src/test/java/com/jobdri/jobdri_api/domain/analysis/dto/worker/AnalysisWorkerContextResponseTest.java (1)

65-87: 🗄️ Data Integrity & Integration | 🟡 Minor | ⚡ Quick win

기존 worker snapshot의 역직렬화 호환성을 추가로 검증하세요.

현재 테스트는 새 Java 생성자와 직렬화만 확인하며, corpusReferences가 없는 기존 snapshot JSON을 새 코드가 읽는지는 검증하지 않습니다. rolling deploy나 retry 중 기존 snapshot을 읽을 수 있으므로, 누락된 필드가 빈 목록으로 정규화되고 similarJobPostings가 유지되는 테스트를 추가하세요.

As per path instructions, src/test/**/*.java에서는 비동기 재시도·실패 복구 경로의 회귀 위험을 확인해야 합니다.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@src/test/java/com/jobdri/jobdri_api/domain/analysis/dto/worker/AnalysisWorkerContextResponseTest.java`
around lines 65 - 87, 기존 생성자 테스트에 추가하여, corpusReferences 필드가 없는 기존 worker
snapshot JSON을 objectMapper로 AnalysisWorkerContextResponse에 역직렬화하는 검증을 작성하세요.
역직렬화 후 corpusReferences가 빈 목록으로 정규화되고 기존 similarJobPostings 값이 그대로 유지되는지 확인하며,
oldConstructorDefaultsSimilarJobPostingsToEmptyList와
AnalysisWorkerContextResponse를 기준으로 관련 테스트를 확장하세요.

Source: Path instructions

src/test/java/com/jobdri/jobdri_api/domain/analysis/service/async/AnalysisWorkerBridgeServiceTest.java (1)

169-192: 🗄️ Data Integrity & Integration | 🟡 Minor | ⚡ Quick win

curated corpus context의 snapshot 재사용 경로를 테스트하세요.

현재 변경된 테스트는 최초 getContext의 성공 응답만 확인합니다. 이후 재호출에서 snapshot의 동일한 corpusReferences를 반환하는지, 완료 시에도 동일 입력으로 finalize하는지 검증하지 않아 context 재조회·retry 회귀를 놓칠 수 있습니다.

As per path instructions, src/test/**/*.java에서는 비동기 재시도와 멱등성 같은 경로를 충분히 검증해야 합니다.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@src/test/java/com/jobdri/jobdri_api/domain/analysis/service/async/AnalysisWorkerBridgeServiceTest.java`
around lines 169 - 192, Extend the getContext test to invoke the context
retrieval again and verify the snapshot’s identical corpusReferences are reused
on the retry. Also exercise completion/finalization and verify it receives the
same original context input, covering the curated corpus snapshot’s idempotent
reuse path and avoiding assertions limited to the initial successful response.

Source: Path instructions

src/main/java/com/jobdri/jobdri_api/domain/analysis/service/core/AnalysisService.java (1)

180-217: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

답변 존재 여부 검증 로직이 두 곳에서 다른 방식으로 중복됩니다.

새로 추가된 validateAnsweredQuestions(209-216, isEmpty() 체크)와 기존 validateAnalysisRequest(116-129, anyMatch 체크)가 동일한 "답변 1개 이상 필요" 규칙을 서로 다른 구현으로 검증하고 있습니다. 한쪽만 수정되면 두 경로의 검증 기준이 어긋날 위험이 있습니다.

♻️ 공통 헬퍼로 통합 제안

     public void validateAnalysisRequest(User user, Long mockApplyId) {
         MockApply mockApply = getOwnedMockApply(user, mockApplyId);
         List<Question> questions = questionRepository.findAllByMockApplyIdOrderByIdAsc(mockApply.getId());
-        boolean hasAnsweredQuestion = questions.stream()
-                .anyMatch(question -> StringUtils.hasText(question.getAnswer()));
-
-        if (!hasAnsweredQuestion) {
-            throw new GeneralException(
-                    GeneralErrorCode.INVALID_PARAMETER,
-                    "분석할 자소서 답변이 1개 이상 필요합니다."
-            );
-        }
+        List<Question> answeredQuestions = questions.stream()
+                .filter(question -> StringUtils.hasText(question.getAnswer()))
+                .toList();
+        validateAnsweredQuestions(answeredQuestions);
     }

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@src/main/java/com/jobdri/jobdri_api/domain/analysis/service/core/AnalysisService.java`
around lines 180 - 217, 통합 분석 요청 흐름의 validateAnalysisRequest와
prepareAnalysisExecution에서 중복된 답변 존재 검증을 하나의 공통 헬퍼로 통합하세요. 기존의 “답변이 1개 이상 필요” 예외
메시지와 오류 코드를 유지하고, 두 경로가 동일한 검증 기준을 사용하도록 validateAnsweredQuestions 또는 적절한 공통
메서드를 재사용하세요.